Skip to content

Deprecate kafka options not available from 9.4.0#6309

Open
khushijain21 wants to merge 3 commits intoelastic:mainfrom
khushijain21:kafka-deprecate
Open

Deprecate kafka options not available from 9.4.0#6309
khushijain21 wants to merge 3 commits intoelastic:mainfrom
khushijain21:kafka-deprecate

Conversation

@khushijain21
Copy link
Copy Markdown
Contributor

@khushijain21 khushijain21 commented May 6, 2026

Summary

We have switched kafka client library from sarama to franz-go as we transitioned from beats' to beatreceivers starting in 9.4.0. This PR deprecated certain options that are no longer applicable/necessary

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

@khushijain21 khushijain21 requested a review from a team as a code owner May 6, 2026 04:32
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Vale Linting Results

Summary: 1 warning found

⚠️ Warnings (1)
File Line Rule Message
reference/fleet/kafka-output.md 16 Elastic.Spelling 'Sarama' is a possible misspelling.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

🔍 Preview links for changed docs

@vishaangelova
Copy link
Copy Markdown
Member

I added my suggestions in 7a6d710. What do you think @khushijain21?

Screenshot 2026-05-06 at 14 59 51

and

Screenshot 2026-05-06 at 14 59 06

Copy link
Copy Markdown
Member

@vishaangelova vishaangelova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added my suggestions in 7a6d710. If you agree with the changes, feel free to merge. :)

@khushijain21
Copy link
Copy Markdown
Contributor Author

cc: @cmacknz can you take a look at this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants